Skip to content

Add support for linux/arm/v7 platform#772

Merged
Viren070 merged 2 commits intoViren070:mainfrom
CKGrafico:patch-1
Mar 1, 2026
Merged

Add support for linux/arm/v7 platform#772
Viren070 merged 2 commits intoViren070:mainfrom
CKGrafico:patch-1

Conversation

@CKGrafico
Copy link
Contributor

@CKGrafico CKGrafico commented Feb 28, 2026 •
edited by coderabbitai bot
Loading

Make compatible with
linux/arm/v7

Summary by CodeRabbit

  • Chores
    • Docker image builds now use environment-driven platform selection so release channel determines target architectures.
    • Stable channel builds now include an additional ARM v7 platform, widening device compatibility and improving availability of prebuilt images for more hardware.

@CKGrafico
Add support for linux/arm/v7 platform
9656360 
Make compatible with 	
linux/arm/v7
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 28, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9656360 and b28d93a.

📒 Files selected for processing (1)
  • .github/workflows/deploy-docker.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/deploy-docker.yml

Walkthrough

The pull request updates the GitHub Actions Docker deployment workflow to set Buildx platforms dynamically via a BUILD_PLATFORMS environment variable based on the CHANNEL value: when CHANNEL is stable it sets linux/amd64,linux/arm64,linux/arm/v7; otherwise it sets linux/amd64,linux/arm64. The docker/build-push-action now uses this variable.

Changes

Cohort / File(s) Summary
Docker Deployment Configuration
.github/workflows/deploy-docker.yml		 Introduce a step that sets BUILD_PLATFORMS conditionally from CHANNEL (stable includes linux/arm/v7, other channels omit it). Replace hard-coded platform list in docker/build-push-action with the BUILD_PLATFORMS env var.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I nibble envs and tweak a line,
If stable's set, V7 joins the sign,
Otherwise two platforms dance and play,
Builds hop smoothly, come what may! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarises the main change: adding support for the linux/arm/v7 platform to the Docker build workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Viren070 Viren070 merged commit 88e83bb into Viren070:main Mar 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CKGrafico @Viren070